Skip to content

[Gtk4] Fix ToolItem image size by using GtkPicture instead of GtkImage#3328

Merged
akurtakov merged 1 commit into
eclipse-platform:masterfrom
akurtakov:toosmallimage
Jun 1, 2026
Merged

[Gtk4] Fix ToolItem image size by using GtkPicture instead of GtkImage#3328
akurtakov merged 1 commit into
eclipse-platform:masterfrom
akurtakov:toosmallimage

Conversation

@akurtakov
Copy link
Copy Markdown
Member

GTK4 works in logical points while Cairo works in pixels, causing images to appear smaller at HiDPI scaling. Using GtkPicture (instead of GtkImage)
correctly maps the pixbuf to the right display size.

This is the same fix applied to Button
(#3323 ) and Label
(#2766 ) for the same reason.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 26, 2026

Test Results (linux)

   94 files  ±0     94 suites  ±0   14m 24s ⏱️ -23s
4 598 tests ±0  4 374 ✅ ±0  224 💤 ±0  0 ❌ ±0 
3 386 runs  ±0  3 311 ✅ ±0   75 💤 ±0  0 ❌ ±0 

Results for commit 411090a. ± Comparison against base commit 81da08f.

♻️ This comment has been updated with latest results.

Copy link
Copy Markdown
Contributor

@jonahgraham jonahgraham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved (for when master reopens)

@akurtakov akurtakov force-pushed the toosmallimage branch 10 times, most recently from 7f5e473 to b2c0025 Compare May 31, 2026 21:19
@akurtakov akurtakov marked this pull request as draft May 31, 2026 21:27
GTK4 works in logical points while Cairo works in pixels, causing images
to appear smaller at HiDPI scaling. Using GtkPicture (instead of
GtkImage)
correctly maps the pixbuf to the right display size.

This is the same fix applied to Button
(eclipse-platform#3323 )
and Label
(eclipse-platform#2766
) for the same reason.
@akurtakov akurtakov marked this pull request as ready for review May 31, 2026 21:57
@akurtakov akurtakov merged commit 80547c4 into eclipse-platform:master Jun 1, 2026
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants